Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow people to only be extra people in a Zulip group #1124

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

rylev
Copy link
Member

@rylev rylev commented Nov 27, 2023

Previously we required everyone included in Zulip user group to be a member of a team. This lifts that restriction.

This fixes the issues encountered in #1120 and #952

@rylev
Copy link
Member Author

rylev commented Dec 7, 2023

ping @Mark-Simulacrum

Mark-Simulacrum
Mark-Simulacrum previously approved these changes Dec 7, 2023
src/validate.rs Outdated Show resolved Hide resolved
Previously we required everyone included in Zulip user group to be a
member of a team. This lifts that restriction.

Signed-off-by: Ryan Levick <[email protected]>
@Mark-Simulacrum Mark-Simulacrum merged commit 8426c92 into rust-lang:master Dec 15, 2023
1 check passed
@rylev rylev deleted the allow-extra-zulip-people branch December 15, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants